Skip to content

updated valkey version - #326

Open
Bloodraven21 wants to merge 1 commit into
valkey-io:mainfrom
Bloodraven21:hotfix/update-valkey-version
Open

updated valkey version#326
Bloodraven21 wants to merge 1 commit into
valkey-io:mainfrom
Bloodraven21:hotfix/update-valkey-version

Conversation

@Bloodraven21

@Bloodraven21 Bloodraven21 commented Jul 22, 2026

Copy link
Copy Markdown
Collaborator

This PR closes <Issue #>

Summary

Updated the valkey version from 9.0.0 -> 9.1.1

Features / Behaviour Changes

just the valkey image change no feat/behaviour change

Implementation

Limitations

Testing

Checklist

Before submitting the PR make sure the following are checked:

  • This Pull Request is related to one issue.
  • Commit message explains what changed and why
  • Tests are added or updated.
  • Documentation files are updated.
  • I have run pre-commit locally (pre-commit run --all-files or hooks on commit)

Signed-off-by: Ishan Jain <ishanij10115@gmail.com>
@greptile-apps

greptile-apps Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR updates the default Valkey version from 9.0.0 to 9.1.1. The main changes are:

  • Updates the operator’s default server image.
  • Aligns unit and end-to-end test images.
  • Updates the private-registry documentation example.

Confidence Score: 5/5

This looks safe to merge.

No blocking issues found in the changed code. Image references are updated consistently. The generation-tracking test still performs a real spec change.

T-Rex T-Rex Logs

What T-Rex did

  • The test was run against valkey/valkey:9.0.0 and passed, establishing a baseline for the validation contract.
  • The test was re-run after updating to valkey/valkey:9.1.1 and passed, validating the post-update contract.
  • Temporary diagnostic test logging was removed after capture and the changed source files have no uncommitted validation edits.

View all artifacts

T-Rex Ran code and verified through T-Rex

Important Files Changed

Filename Overview
internal/controller/valkeycluster_controller.go Updates the default Valkey image to 9.1.1.
test/e2e/valkeynode_test.go Updates the generation-tracking patch to use Valkey 9.1.1; the patch still changes the stored image from omitted to explicit.
test/e2e/e2e_suite_test.go Updates the Valkey client image used by end-to-end tests.
internal/controller/valkeycluster_controller_test.go Aligns cluster reconciliation fixtures with Valkey 9.1.1.
internal/controller/valkeynode_controller_test.go Aligns workload rollout fixtures with Valkey 9.1.1.
internal/controller/valkeynode_resources_test.go Updates node resource fixtures and image assertions to Valkey 9.1.1.
docs/valkeycluster.md Updates the private-registry example to Valkey 9.1.1.

Reviews (1): Last reviewed commit: "updated valkey version" | Re-trigger Greptile

Comment thread internal/controller/valkeycluster_controller.go
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants